Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: APPS-2982 Remove excess padding values #620

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

tinuola
Copy link
Contributor

@tinuola tinuola commented Sep 25, 2024

Connected to APPS-2982

Component Created: TwoColLayoutWStickySideBar.vue

Notes:

  • Removed padding to the left and right of the component.

Before:
event-series-page-fixes

After:
twoColStickySidebar-fixed-padding

tablet-mob

Checklist:

  • I checked that it is working locally in the dev server
  • I checked that it is working locally in the storybook
  • I checked that it is working locally in the
    library-website-nuxt dev server
  • I added a screenshot of it working
  • UX has reviewed and approved this
  • I requested a PR review from the Dev team
  • I used a conventional commit message
  • I assigned myself to this PR

@tinuola tinuola added the fix label Sep 25, 2024
@tinuola tinuola self-assigned this Sep 25, 2024
Copy link
Contributor

Percy Screenshots

In order to conserve our percy screenshot allowance, percy is not configured to run automatically. Please make sure the PR is ready and all other checks are passing, then start it manually:

  1. Visit https://github.com/UCLALibrary/ucla-library-website-components/actions/workflows/percy.yml 2. Click the 'Run workflow' button in the blue bar. 3. Select the correct branch for this PR and click 'Run workflow' again to confirm.

Copy link
Contributor

github-actions bot commented Sep 25, 2024

@github-actions github-actions bot temporarily deployed to storybook--pull_request-620 September 25, 2024 18:52 Inactive
Copy link
Contributor

@farosFreed farosFreed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a quick question, but I am approving it otherwise. Looks good!

@tinuola tinuola merged commit d895d3c into main Sep 25, 2024
13 checks passed
@tinuola tinuola deleted the APPS-2982_fix-twoColSidebar-padding branch September 25, 2024 23:07
pghorpade pushed a commit that referenced this pull request Sep 25, 2024
## [3.22.1](v3.22.0...v3.22.1) (2024-09-25)

### Bug Fixes

* APPS-2982 Remove excess padding values ([#620](#620)) ([d895d3c](d895d3c))
@pghorpade
Copy link
Contributor

🎉 This PR is included in version 3.22.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

pghorpade pushed a commit that referenced this pull request Sep 27, 2024
* fix: remove excess padding values

* adjust responsiveness of fix

---------

Co-authored-by: tinuola <[email protected]>
pghorpade pushed a commit that referenced this pull request Sep 27, 2024
## [3.22.1](v3.22.0...v3.22.1) (2024-09-25)

### Bug Fixes

* APPS-2982 Remove excess padding values ([#620](#620)) ([d895d3c](d895d3c))
pghorpade added a commit that referenced this pull request Sep 27, 2024
* feat: set ref ismobile to false and update it in onmounted client side

* fix: hydration error and undefined errors by adding watchers

* fix: replace globalstore with useWindowSize from vueuse , also revert iconwithlink and section-wrapper  changes not needed in this PR and add watcher to update layout data when nuxt needs to work with craft draft previews

* feat: APPS-2943 Add initialTab setting option to TabToggle (#611)

* wip: add prop for initialTab selection

* refactor glider animation to work with setting intial tab position

* remove margin value on glider/offset

* add story for initialTab prop

* refactor: tab glider animation

* task: update code documentation

---------

Co-authored-by: tinuola <[email protected]>

* chore(release): set `package.json` to 3.19.0 [skip ci]

# [3.19.0](v3.18.0...v3.19.0) (2024-09-17)

### Features

* APPS-2943 Add initialTab setting option to TabToggle ([#611](#611)) ([aadedac](aadedac))

* feat: APPS-2945 FTVA Variation of SectionTeaserList (#613)

* add separated ftva style files

* remove border bottom just for FTVA

* make /plan-your-visit the default URL for the Plan Your Visit button in the BlockInfo component

* uAdd a SectionWrapper story for TabList component that uses SectionTeaserList

* fixes

* fixes

* section-header

* linting

* fix past events and csectionteasercard spacing

* fix gap

* revert default/_section-teaser-card.scss

* update changes to blockCardThreeColumn

* default styles for section-teaser-list

* default styles for section-teaser-list

* delete commented out code

* move styling to wrapper

* update vertical cards

* fix syntax

* chore(release): set `package.json` to 3.20.0 [skip ci]

# [3.20.0](v3.19.0...v3.20.0) (2024-09-20)

### Features

* APPS-2945 FTVA Variation of SectionTeaserList ([#613](#613)) ([29e04f0](29e04f0))

* feat: APPS-2946 Create Two Column Layout component (#614)

* feat: begin sidebar structure, start drawio docs

* feat: more progress side

* feat: progress on 2col, stories

* chore: finish stories, tests

* chore: cleanup, documentation

* fix: double padding

* fix: inconsistent spacing when card-meta is above sidebar

---------

Co-authored-by: Jess Divers <[email protected]>

* chore(release): set `package.json` to 3.21.0 [skip ci]

# [3.21.0](v3.20.0...v3.21.0) (2024-09-24)

### Features

* APPS-2946 Create Two Column Layout component ([#614](#614)) ([cc5d52d](cc5d52d))

* fix: APPS-2947 Update NavBreadcrumb logic to display route title or prop title (#616)

* wip: refactor breadcrumb logic

* wip: update breadcrumb stories

* task: clean up code

* fix: add extra logic for parent breadcrumb icon on mobile

* update code documentation

* update ref, function naming

* fix: update ref name in template

* fix: remove collapsed class when group is expanded

---------

Co-authored-by: tinuola <[email protected]>
Co-authored-by: Parinita Mulak <[email protected]>

* chore(release): set `package.json` to 3.21.1 [skip ci]

## [3.21.1](v3.21.0...v3.21.1) (2024-09-24)

### Bug Fixes

* APPS-2947 Update NavBreadcrumb logic to display route title or prop title ([#616](#616)) ([cba8af6](cba8af6))

* feat: APPS-2957 Create FTVA Pullquote (#617)

* feat: ftva pull-quote

* chore: lint

* fix: font weight, non italic

* fix: bolder bold inside pullquote

* fix: font weight

* fix: add 12px

---------

Co-authored-by: Jess Divers <[email protected]>
Co-authored-by: Parinita Mulak <[email protected]>

* chore(release): set `package.json` to 3.22.0 [skip ci]

# [3.22.0](v3.21.1...v3.22.0) (2024-09-24)

### Features

* APPS-2957 Create FTVA Pullquote ([#617](#617)) ([65577e8](65577e8))

* fix: APPS-2982 Remove excess padding values (#620)

* fix: remove excess padding values

* adjust responsiveness of fix

---------

Co-authored-by: tinuola <[email protected]>

* chore(release): set `package.json` to 3.22.1 [skip ci]

## [3.22.1](v3.22.0...v3.22.1) (2024-09-25)

### Bug Fixes

* APPS-2982 Remove excess padding values ([#620](#620)) ([d895d3c](d895d3c))

* chore: APPS-2962 move styles from page to story (#619)

* chore: move styles from page to story

* fix: tweak gap for 2 cards

---------

Co-authored-by: Jess Divers <[email protected]>
Co-authored-by: Tinuola 'Tinu' Awopetu <[email protected]>

* feat: Apps-2951 card meta changes for article detail page (#618)

* feat: card-meta date-created prop

* feat: ftvaArticle styles

* fix: linter, tweaks

* fix: styles based on UX feedback

* fix: add globalstore to story for sharebutton

* fix: mistake in story

---------

Co-authored-by: Jess Divers <[email protected]>

* chore(release): set `package.json` to 3.23.0 [skip ci]

# [3.23.0](v3.22.1...v3.23.0) (2024-09-27)

### Features

* Apps-2951 card meta changes for article detail page ([#618](#618)) ([ace6bfe](ace6bfe))

* fix: hydration error and undefined errors by adding watchers

* fix: lint errors

---------

Co-authored-by: Tinuola 'Tinu' Awopetu <[email protected]>
Co-authored-by: tinuola <[email protected]>
Co-authored-by: semantic-release-bot <[email protected]>
Co-authored-by: JenDiamond <[email protected]>
Co-authored-by: Jess <[email protected]>
Co-authored-by: Jess Divers <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants